Repository navigation
Chart: Update DiscretePointsAggregation Demo Desc. - #36249
Open
arman-boyakhchyan wants to merge 3 commits into
Open
arman-boyakhchyan wants to merge 3 commits into
arman-boyakhchyan wants to merge 3 commits into
Conversation
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The description implies gray maximum-reference bars are visible in Sum mode, but taller overlapping sum bars hide them.
1 open issue
What changed in this PR
Updates the Chart discrete-point aggregation demo description to explain its oil-production data, overlapping bars, and aggregation controls. No runtime code changes.
Changes:
- Replaces generic setup steps with demo-specific explanations of aggregation, tooltips, and available methods.
- Clarifies common aggregation settings and the gray series’ maximum-value override.
| File | Description |
|---|---|
| apps/demos/Demos/Charts/DiscretePointsAggregation/description.md | Rewrites the demo explanation and configuration guidance. |
Review effort: Balanced
khamlyuk
reviewed
Nov 11, 2026
|
|
||
| - Individual series | ||
| Assign `true` to the [series](/Documentation/ApiReference/UI_Components/dxChart/Configuration/series/).[aggregation](/Documentation/ApiReference/UI_Components/dxChart/Configuration/series/aggregation/).[enabled](/Documentation/ApiReference/UI_Components/dxChart/Configuration/series/aggregation/#enabled) property. | ||
| This demo displays monthly crude oil production in six US states. The Chart creates a series for each year, so every series contains twelve values per state. Without aggregation, the Chart draws these values as twelve overlapping bars in the same position. Aggregation replaces them with a single bar. |
Contributor
There was a problem hiding this comment.
Currently, sounds like a bug. Correct me if my edit is wrong.
Suggested change
| This demo displays monthly crude oil production in six US states. The Chart creates a series for each year, so every series contains twelve values per state. Without aggregation, the Chart draws these values as twelve overlapping bars in the same position. Aggregation replaces them with a single bar. | |
| This demo displays monthly crude oil production in six US states. The Chart creates a series for each year, so that every series contains twelve values per state. Without aggregation, the Chart draws these values as twelve stacked bars in the same position. Aggregation replaces them with a single bar. |
Contributor
Author
There was a problem hiding this comment.
"Overlapping" is the correct word here, "Stacked" bars are a separate concept: https://js.devexpress.com/Documentation/ApiReference/UI_Components/dxChart/Series_Types/StackedBarSeries/
|
|
||
| In this demo, data aggregation is enabled for all series. | ||
| - **Aggregation enabled** | ||
| Clear this checkbox to display source data. All twelve monthly bars overlap and look like a single bar as tall as the highest monthly value. The tooltip displays the month for the bar under the mouse pointer, and this month changes as you move the pointer along the bar. |
Contributor
There was a problem hiding this comment.
How about this?
Suggested change
| Clear this checkbox to display source data. All twelve monthly bars overlap and look like a single bar as tall as the highest monthly value. The tooltip displays the month for the bar under the mouse pointer, and this month changes as you move the pointer along the bar. | |
| Uncheck this checkbox to display original (source) data as stacked bars. The tooltip displays the bar's month on hover. |
Contributor
Author
There was a problem hiding this comment.
That is not quite accurate. Each bar you see in this demo is in fact 12 overlapping bars (one for each month of the year). Tooltips are displayed for individual months only (when aggregation is disabled)
| - **Method** | ||
| Select the value that aggregated bars display: average, sum, or minimum. Gray bars in the background display the monthly maximum for comparison when average or minimum is selected. | ||
|
|
||
| You can apply common aggregation settings to all series. Activate [aggregation](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/).[enabled](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/#enabled) in [commonSeriesSettings](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/) and specify an aggregation [method](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/#method). Individual [series](/Documentation/ApiReference/UI_Components/dxChart/Configuration/series/aggregation/) settings override these common settings. In this demo, the gray series always use the *"max"* aggregation method. |
Contributor
There was a problem hiding this comment.
Suggested change
| You can apply common aggregation settings to all series. Activate [aggregation](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/).[enabled](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/#enabled) in [commonSeriesSettings](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/) and specify an aggregation [method](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/#method). Individual [series](/Documentation/ApiReference/UI_Components/dxChart/Configuration/series/aggregation/) settings override these common settings. In this demo, the gray series always use the *"max"* aggregation method. | |
| You can apply common aggregation settings to all series. Activate [aggregation](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/).[enabled](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/#enabled) option in [commonSeriesSettings](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/) and specify an aggregation [method](/Documentation/ApiReference/UI_Components/dxChart/Configuration/commonSeriesSettings/aggregation/#method). Individual [series](/Documentation/ApiReference/UI_Components/dxChart/Configuration/series/aggregation/) settings override these common settings. |
Contributor
Author
There was a problem hiding this comment.
The "gray series" here refers to the background bar, should we not describe it here?
Contributor
There was a problem hiding this comment.
We've mentioned this in the previous paragraph. I don't think we should repeat the idea.
Co-authored-by: Elena Khamliuk <80813840+khamlyuk@users.noreply.github.com> Signed-off-by: Arman Boyakhchyan <arman.boyakhchyan@devexpress.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.